gesturerotate: Just return a double in get_angle_delta()
authorCarlos Garnacho <carlosg@gnome.org>
Mon, 26 May 2014 12:29:17 +0000 (14:29 +0200)
committerCarlos Garnacho <carlosg@gnome.org>
Tue, 27 May 2014 15:47:12 +0000 (17:47 +0200)
Whether the gesture is active or recognized should be checked before in
callers, or just not used naively.

demos/gtk-demo/gestures.c
gtk/gtkgesturerotate.c
gtk/gtkgesturerotate.h

index 77da33582493ea94964ee50aa56816a09d671a21..e9f80fcb346c4a3996c2a02c14eea0b2843b1ef3 100644 (file)
@@ -93,8 +93,8 @@ drawing_area_draw (GtkWidget *widget,
 
       cairo_save (cr);
 
-      if (gtk_gesture_rotate_get_angle_delta (GTK_GESTURE_ROTATE (rotate), &angle))
-        cairo_matrix_rotate (&matrix, angle);
+      angle = gtk_gesture_rotate_get_angle_delta (GTK_GESTURE_ROTATE (rotate));
+      cairo_matrix_rotate (&matrix, angle);
 
       if (gtk_gesture_zoom_get_scale_delta (GTK_GESTURE_ZOOM (zoom), &scale))
         cairo_matrix_scale (&matrix, scale, scale);
index ec8b12ae1ce7611b8fcd9d9844ac0629f921d3ea..8675125bb67d8d29ed409f98e0c9071c9188bfef 100644 (file)
@@ -196,19 +196,16 @@ gtk_gesture_rotate_new (GtkWidget *widget)
 /**
  * gtk_gesture_rotate_get_angle_delta:
  * @gesture: a #GtkGestureRotate
- * @delta: (out) (transfer none): angle delta
  *
- * If @gesture is active, this function returns %TRUE and fills
- * in @delta with the angle difference in radians since the
- * gesture was first recognized.
+ * If @gesture is active, this function returns the angle difference
+ * in radians since the gesture was first recognized.
  *
- * Returns: %TRUE if @controller is recognizing a rotate gesture
+ * Returns: the angle delta in radians
  *
  * Since: 3.14
  **/
-gboolean
-gtk_gesture_rotate_get_angle_delta (GtkGestureRotate *gesture,
-                                    gdouble          *delta)
+gdouble
+gtk_gesture_rotate_get_angle_delta (GtkGestureRotate *gesture)
 {
   GtkGestureRotatePrivate *priv;
   gdouble angle;
@@ -216,12 +213,9 @@ gtk_gesture_rotate_get_angle_delta (GtkGestureRotate *gesture,
   g_return_val_if_fail (GTK_IS_GESTURE_ROTATE (gesture), FALSE);
 
   if (!_gtk_gesture_rotate_get_angle (gesture, &angle))
-    return FALSE;
+    return 0;
 
   priv = gtk_gesture_rotate_get_instance_private (gesture);
 
-  if (delta)
-    *delta = angle - priv->initial_angle;
-
-  return TRUE;
+  return angle - priv->initial_angle;
 }
index 803355d053a5560bf8488514c5f157ba2e31f2fc..4db84e431ed6d68532016c761f0a418f07af2023 100644 (file)
@@ -46,8 +46,7 @@ GDK_AVAILABLE_IN_3_14
 GtkGesture * gtk_gesture_rotate_new             (GtkWidget        *widget);
 
 GDK_AVAILABLE_IN_3_14
-gboolean     gtk_gesture_rotate_get_angle_delta (GtkGestureRotate *gesture,
-                                                 gdouble          *delta);
+gdouble      gtk_gesture_rotate_get_angle_delta (GtkGestureRotate *gesture);
 
 G_END_DECLS